Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Set identity during admin init #181

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Set identity during admin init #181

merged 1 commit into from
Oct 25, 2023

Conversation

xinaxu
Copy link
Collaborator

@xinaxu xinaxu commented Oct 24, 2023

Follow-up of
#177

Motion container starts after other Singularity container starts hence setting identity inside motion container does not take effect until the user restart the docker compose.

To address above issue, we can set the identity to Motion during Admin init script.

Tested end to end and see data reflected on Dashboard
https://dataprograms.grafana.net/d/f49ba403-a767-4243-af44-607dec1e7100/usage?orgId=1

fixes #174

@codecov-commenter
Copy link

Codecov Report

Merging #181 (8dc0f98) into main (bb873c7) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #181   +/-   ##
=======================================
  Coverage   30.95%   30.95%           
=======================================
  Files           5        5           
  Lines         756      756           
=======================================
  Hits          234      234           
  Misses        485      485           
  Partials       37       37           

@xinaxu xinaxu requested a review from masih October 24, 2023 23:30
@xinaxu xinaxu merged commit 28811ff into main Oct 25, 2023
7 checks passed
@xinaxu xinaxu deleted the xinaxu/set-identity branch October 25, 2023 17:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Motion telemetry through Singularity
3 participants